Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Decoupling mongo db dependencies to allow adding postgreSQL #213

Merged
merged 16 commits into from
Jun 21, 2024
Merged

Decoupling mongo db dependencies to allow adding postgreSQL #213

merged 16 commits into from
Jun 21, 2024

Conversation

DavidGershony
Copy link
Contributor

No description provided.

DavidGershony and others added 15 commits June 10, 2024 13:13
@DavidGershony
Moved mongo service registration into mongo namespace
5cf9bd1
@DavidGershony
Removed mongo dependency from storage operations class
8e956c1
@DavidGershony
Moved the rich list scanning into mongo (this is actually only releva…
3703b90 
…nt for address based blockchains unlike BTC)
@DavidGershony
The block indexing sync task is dedicated to mongo db and has been mo…
0f7c93c 
…ved into the mongo namespace
@DavidGershony
Moved Mongo driver dependency into mongo namespace
c936800
@DavidGershony
Removed stats syncer and stats controller dependency on mongo namespace
451391b
@DavidGershony
Removed mongo dependency from insight controller
f97b1f3
@DavidGershony
Removed mongo dependency from query controller
a10d5c7
@DavidGershony
Removed mongo dependency from QueryOrphanBlock.cs
1ff847e
@DavidGershony
Removed mongo dependency from InsertStats class
752147b
@DavidGershony
Commented out unused code that has a dependency on mongo namespace
1e0c71e
@DavidGershony
Renamed settings properties from prefix mongo to db
382c309
@DavidGershony
Removed mongo dependency from core tasks
65b9f08
@DavidGershony
Fixed text in controller
1a6be60
@DavidGershony
Decoupled the storage batch from Mongo db dependency
2851a26
dangershony
dangershony reviewed Jun 19, 2024
View reviewed changes
Copy link
Member

@dangershony dangershony left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR looks ok to me, did you manage to test starting it from zero? (building the indexes)

@DavidGershony DavidGershony merged commit e27a2c9 into master Jun 21, 2024
3 checks passed
theDRB123 added a commit to theDRB123/blockcore-indexer that referenced this pull request Jun 21, 2024
@theDRB123
Merge pull request #1 from block-core/master
7514b29 
Decoupling mongo db dependencies to allow adding postgreSQL (block-core#213)
theDRB123 added a commit to theDRB123/blockcore-indexer that referenced this pull request Jun 21, 2024
@theDRB123
Revert "Decoupling mongo db dependencies to allow adding postgreSQL (b…
1b943ce 
…lock-core#213)"
theDRB123 added a commit to theDRB123/blockcore-indexer that referenced this pull request Jun 21, 2024
@theDRB123
Merge pull request #2 from theDRB123/revert-1-master
dd463e6 
Revert "Decoupling mongo db dependencies to allow adding postgreSQL (block-core#213)"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dangershony dangershony dangershony approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@DavidGershony @dangershony